Skip to content

Add property for given names#157

Closed
fdesoye wants to merge 2 commits into
derek73:masterfrom
fdesoye:given_names
Closed

Add property for given names#157
fdesoye wants to merge 2 commits into
derek73:masterfrom
fdesoye:given_names

Conversation

@fdesoye

@fdesoye fdesoye commented Sep 30, 2024

Copy link
Copy Markdown
Contributor

Just like surnames, this feature will give you a list of all given names, i.e. all names except titles and last name.

@derek73 derek73 added this to the v1.3.0 milestone Jun 29, 2026
derek73 added a commit that referenced this pull request Jun 29, 2026
* feat: add given_names attribute aggregating first and middle names (closes #157)

Adds given_names (and given_names_list), the first name followed by all
middle names, mirroring the existing surnames attribute (middle + last).
Reimplemented fresh on current master from the approach in #157, which was
based on a pre-refactor codebase and had drifted.

Co-Authored-By: fdesoye <fdesoye@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* test: add empty-path and first-only tests for given_names; document pattern in AGENTS.md

Covers the `or self.C.empty_attribute_default` branch (untested before this commit)
and the no-middle boundary case. Also adds a note to AGENTS.md reminding future
contributors to include an empty-path test for any new aggregate property.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: fdesoye <fdesoye@users.noreply.github.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
@derek73 derek73 closed this in #180 Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants